home *** CD-ROM | disk | FTP | other *** search
/ Atari Mega Archive 1 / Atari Mega Archive - Volume 1.iso / lists / mint / l_1599 / 1231 < prev    next >
Encoding:
Internet Message Format  |  1994-08-27  |  2.2 KB
  1. Subject: Re: Mint 1.10: Bug in Fselect()
  2. Date: Mon, 21 Mar 94 22:13:50 CET
  3. From: Juergen Lock <nox@jelal.north.de>
  4. In-Reply-To: <9403210822.AA05825@hera.rbi.informatik.uni-frankfurt.de>; from "Kay Roemer" at Mar 21, 94 9:22 am
  5. Message-Id: <9403212113.AA00065@jelal.north.de>
  6.  
  7. Kay Roemer writes:
  8. > This makes Fselect interruptible by signals. This feature was present
  9. > in MiNT 1.09, but was lost in 1.10.
  10. > 
  11. > Without this patch inetd and some other programs do not work correctly.
  12. > 
  13. > Kay.
  14. > 
  15. > --8<-------------- cut here ----------------------------
  16. > *** dosfile.c.orig    Mon Mar  7 20:09:06 1994
  17. > --- dosfile.c    Mon Mar  7 20:34:44 1994
  18. > ***************
  19. > *** 1040,1046 ****
  20. >       /* curproc->wait_cond changes when data arrives or the timeout happens */
  21. >           while (curproc->wait_cond == (long)wakeselect) {
  22. >               TRACE(("sleeping in Fselect"));
  23. > !             sleep(SELECT_Q, (long)wakeselect);
  24. >           }
  25. >           spl(sr);
  26. >   
  27. > --- 1040,1049 ----
  28. >       /* curproc->wait_cond changes when data arrives or the timeout happens */
  29. >           while (curproc->wait_cond == (long)wakeselect) {
  30. >               TRACE(("sleeping in Fselect"));
  31. > !             if (sleep(SELECT_Q, (long)wakeselect)) {
  32. > !                 /* interrupted by signals */
  33. > !                 break;
  34. > !             }
  35. >           }
  36. >           spl(sr);
  37. >   
  38. > --8<-------------- cut here ----------------------------
  39.  
  40.  ahaa so my last patch should better be changed like this?
  41.  
  42. Index: dosfile.c
  43. @@ -1045,6 +1045,7 @@
  44.      }
  45.  
  46.      if (count == 0) {
  47. +        ulong onsigs;
  48.          /* OK, now let's set a timeout */
  49.  
  50.          if (timeout) {
  51. @@ -1057,6 +1058,7 @@
  52.          sr = spl7();
  53.  
  54.      /* curproc->wait_cond changes when data arrives or the timeout happens */
  55. +        onsigs = curproc->nsigs;
  56.          while (curproc->wait_cond == (long)wakeselect) {
  57.  #if 0
  58.  /* better not call BIOS with interrupts off, especially not Bconin... :) */
  59. @@ -1068,6 +1070,8 @@
  60.                  check_sigs();
  61.                  sr = spl7();
  62.              }
  63. +            if (onsigs != curproc->nsigs)
  64. +                break;
  65.          }
  66.          spl(sr);
  67.  
  68.  or _must_ check_sigs run at level 7 here?  then stopping the modem1
  69. receiver overruns could get a bit harder...
  70.  
  71.  cheers
  72.     Juergen
  73. -- 
  74. J"urgen Lock / nox@jelal.north.de / UUCP: ..!uunet!unido!uniol!jelal!nox
  75.                                 ...ohne Gewehr
  76. PGP public key fingerprint =  8A 18 58 54 03 7B FC 12  1F 8B 63 C7 19 27 CF DA 
  77.